Skip to content

Fix connection error, again#1743

Open
user202729 wants to merge 3 commits intoopensteno:mainfrom
user202729:connection-reset-2
Open

Fix connection error, again#1743
user202729 wants to merge 3 commits intoopensteno:mainfrom
user202729:connection-reset-2

Conversation

@user202729
Copy link
Member

@user202729 user202729 commented Jun 22, 2025

Should probably use this for a few days to ensure it doesn't happen, will un-draft the PR by then.

So far I haven't been able to reproduce it after the change; but then I don't use the latest version (for now)

Previously a stroke that maps to the following can occasionally reproduce the issue.

{PLOVER:SHELL:xterm -e tmux new bash -c "sleep 0.5s; setxkbmap us; plover --log-level info; bash" &}{PLOVER:QUIT}

The explanation is in the comment.

possible race condition: a ConnectionResetError might be caused
by the previous instance dying just as this instance tries to
connect to it. In that case self._address would have existed
at the creation of controller but now no longer exists.
We ignore the error

Closes #1738

Pull Request Checklist

  • Changes have tests
  • News fragment added in news.d. See documentation for details

@user202729 user202729 force-pushed the connection-reset-2 branch from faf298a to f24d1ef Compare July 17, 2025 12:52
@user202729 user202729 marked this pull request as ready for review July 17, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ConectionResetError, again

1 participant